Guard against selection requestor disappearing

We made a number of unchecked accesses to the selection requestor,
which could cause X errors if the selection requestor vanished
(or we were sent invalid events). Add error traps around all of them.

See https://bugzilla.redhat.com/show_bug.cgi?id=1210814 for an
example of a crash that could be caused by this. (There are about
1000 instances of such a crash in http://retrace.fedoraproject.org
though no linked bug reports specifically talk about cut-and-paste
or DND.)

https://bugzilla.gnome.org/show_bug.cgi?id=756881

iconcache: Remove unused function

window: Name the decoration style

window: Add gtk_widget_class_set_css_name()

This is to replace using class names as CSS names.

window: Refactor function

Move gtk_style_context_save() into the function that sets up the
decoration rendering.

notebook: Remove "prelight-page" class

Instead, just use the "tab:hover" selector

notebook: Remove "active-page" class

Instead, just use the "tab:active" selector

notebook: Remove funciton by folding it into callers

notebook: Refactor internal function

Don't return the state anymore as the function doesn't even look at the
state. Instead, make the callers that need the state query it manually.

notebook: Make tab positions permanent

... on the tab CssNodes.

notebook: Remove last traces of regions

Now that we don't use regions anymore, there's no need to update
anything when the regions change.

notebook: Set reorderable-page style class properly

Set it when the reorderable state changes, not just when rendering.

notebook: Set active state properly

Don't update it on save/restore, actually set it when it changes.

notebook: Set prelight state properly

Don't update it on save/restore, actually set it when it changes.

notebook: Use CssNodes instead of regions

This almost makes CSS work again with notebooks.

notebook: Move style_context_save() into function

This is for future changes.

css: Remove macros that were used only with regions

cssmatcher: Remove matching API for regions

treeview: Use a cssnode instead of regions

This makes Treeview headers work again like before region support was
removed.

API: cssselector: Stop supporting regions

The namespace that belonged to regions is supposed to be used for
CssNode names.

treeviewcolumn: Create button on init

This way, the button is always available and the code doesn't have to
special case this condition.

switch: Port to GtkCssNode

This is a simple port, no code modifications so far other than replacing
gtk_style_context_save() with gtk_style_context_save_to_node().

stylecontext: Add gtk_style_context_save_to_node()

To be used instead of gtk_style_context_save() with persistent nodes.

stylecontext: Add gtk_style_context_add_named()

This is an intermediate step for cssnode introduction. It gives a name
for the node created by saving the style context.

cssmatcher: Marshal name to matcher

... and use it in the node matcher.

Also rename function from _gtk_css_matcher_get_type() to
_gtk_css_matcher_get_name().

cssnode: Add setters/getters for name

cssnodedeclaration: Add possibility to set the name

This is supposed to be a replacement for setting the type. So far, both
options are possible - either will unset the other.

treeview: Remove "row" and "col" regions

They aren't used by the theme and we want to get rid of regions.

Updated POTFILES.in and POTFILES.skip

Update NEWS to mention help overlays

Update POTFILES.in

widget-factory: Add an automatic help overlay

This commit add some more keyboard shortcuts to gtk3-widget-factory,
and adds a help overlay documenting them. This examle uses the
automatic resource loading support in GtkApplication.

Add automatic help overlay support to GtkApplication

When the $(resource_prefix)/gtk/help-overlay.ui resource exists,
load a GtkShortcutsWindow from it for each GtkApplicationWindow,
and set up a win.show-help-overlay action with accels <Primary>F1
and <Primary>? to show it.

gtk-demo: Add a GtkShortcutsWindow demo

This example implements the mockups from the help overlay design,
showing off the various features of GtkShortcutsWindow.

Add some styling for GtkShortcutsWindow

Add some keycap frame around keysyms, and make the stack switcher
buttons rounds. This is very provisional.

Add GtkShortcutsWindow

This is a toplevel window that is tailored towards showing
help for shortcuts in an application. The implementation closely
follows this design: https://wiki.gnome.org/Design/OS/HelpOverlay

This implementation is inspired by earlier work in gnome-builder,
thanks to Christian Hergert.

https://bugzilla.gnome.org/show_bug.cgi?id=756428

builder: Support creating GFile objects

This is useful, e.g. when creating GFileIcon objects.

Don't use g_slist_next in the testsuite

We generally use ->next directly.

Don't use g_slist_next in gtktreeview.c

We generally use ->next directly.

Don't use g_slist_next in gtktreemodel.c

We generally use ->next directly.

Don't use g_slist_next in gtktextview.c

We generally use ->next directly.

Don't use g_slist_next in gtktextlayout.c

We generally use ->next directly.

Don't use g_slist_next in gtktextdisplay.c

We generally use ->next directly.

Don't use g_slist_next in gtktextchild.c

We generally use ->next directly.

Don't use g_slist_next in gtktextbufferrichtext.c

We generally use ->next directly.

Don't use g_slist_next in gtktextbuffer.c

We generally use ->next directly.

Don't use g_slist_next in gtkstock.c

We generally use ->next directly.

Don't use g_slist_next in gtkiconfactory.c

We generally use ->next directly.

Don't use g_slist_next in the x11 backend

We generally use ->next directly.

Don't use g_slist_next in the windows backend

We generally use ->next directly.

Don't use g_slist_next in gdk

We generally just use ->next directly.

Don't use g_list_next in gtk3-demo

We generally just use ->next directly.

Don't use g_list_next in gtkselection.c

We generally access ->next directly.

Don't use g_list_next in gtkmain.c

We generally access ->next directly.

Don't use g_list_next in gtkdialog.c

We generall access ->next directly.

Don't use g_list_next in gtkcontainer.c

We generally access ->next directory.

Don't use g_slist_next in gtktextbtree.c

We generally access ->next directly.

builder: Don't use g_slist_next

We just use direct access to list->next all over the place.

builder: Cosmetic changes

Use an iter instead of g_hash_table_forall.

Cosmetic changes

Line up struct-filling code with the order in the declaration.

Add some more builder parser tests

These tests contain non-canonical and non-existing property names.

builder: Avoid some unnecessary overhead

Only get the class once per object, not once per property.
And don't canonicalize the property name, g_object_class_find_property
does that already.

Updated Basque language

Updated Basque language

widget: fix typo in warning message

wayland: Map windows with tooltip hint as subsurfaces

Tooltips tend to be placed on top of a parent surface with a given
relative coordinate, and without any input focus. So lets map them as
subsurfaces.

https://bugzilla.gnome.org/show_bug.cgi?id=756496

wayland: Clean up code determining how to map a window

Restructure the mapping procedure so that its known up front what the
expected way mapping is to be done (subsurface, popup or stand alone),
and warn if it fails to actually map in such a way (for example a popup
without a parent or device grab, a tooltip without a parent).

https://bugzilla.gnome.org/show_bug.cgi?id=756496

Minor cleanup

Replace a unref-and-unset combination with g_clear_object.

Clean up handling of _gdk_win32_grab_cursor

This is a variable holding a ref to an object, so it is
a great case to use g_set_object and g_clear_object.

 # Please enter the commit message for your changes. Lines starting

W32: Add missing reference taking in gdk_device_virtual_grab()

https://bugzilla.gnome.org/show_bug.cgi?id=756160

Updated Norwegian bokmål translation.

Fixes from Einar Ulvsløkk.

GtkPaned: Fix gtk_paned_set_child_visible calls

Use CHILD1/CHILD2 instead of 0 and 1, always use the same order and
don't check for child NULL-ness, because it will be done in
gtk_paned_set_child_visible anyways.

GtkRevealer: Remove useless _start_animation call

The call just passed priv->target_pos, but _start_animation directly
returns if the given target is priv->target_pos (see line 627).

mir: send touch events

win32: fix warnings about signed/unsigned mismatch

win32: use the same type as the prototype

inspector: remove gtk_text_buffer_get_iter_at_line_index() workaround

The function is now safe to use.

https://bugzilla.gnome.org/show_bug.cgi?id=735341

textbuffer: nicer get_iter functions

Avoid crashes when passing an invalid location to a
gtk_text_buffer_get_iter_at_*() function.

A boolean is returned to know if @iter has been set to the exact
location.

Unit tests are added.

https://bugzilla.gnome.org/show_bug.cgi?id=735341

Updated Slovak translation

Updated Slovak translation

Split off a private header for GtkTextBuffer

This avoids polluting the installed header with private symbols.

Improve text a11y test

Test appending text as well as replacing the entire text.
Insert non-ASCII text to verify offsets are correct.

file chooser: Avoid search interruption

When the search entry is shown, the 'special' nature of
., ~ and / should not trigger the location entry, because
that interrupts the search and is likely not what the
user intended.

https://bugzilla.gnome.org/show_bug.cgi?id=756505

entry completion: Only grab the device if we have one

When the entry completion is popped up from a timeout, we may
not have a device. In that case, don't call gdk_device_grab,
do avoid criticals.

scrolled window: Protect against nameless devices

It seems that gdk_device_get_name() can return NULL.
We should not crash if that happens.

https://bugzilla.gnome.org/show_bug.cgi?id=756625

gtkplacesview: align spinner with header label

Use the box margin top instead of the label margin top,
so the spinner remains aligned with the header label.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: plug leak

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: remove hover color from rows

Since other views are not using hover neither

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: add a clear button to address entry

So it allows a quick way to clear the entry.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: rotate server list icon on toggled

Disclosure triangles are usually used pointing down, however
in this case the popover spawns in the upper direction, which
makes it odd looking.
Instead of pointing always down or up, point down when not toggled and
animate a rotation when toggled.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: tweak ui to allow more server rows

Following design guidance, reduce row height and increase
popover height so the user is allowed to see more than 3
rows.

https://bugzilla.gnome.org/show_bug.cgi?id=756568

gtkplacesview: improve heuristics for external drives

Following the sidebar on commit b0989b190df, improve the way
we check when a drive is external or not.

https://bugzilla.gnome.org/show_bug.cgi?id=756589

gtkplacesview: remove dead code

This is checked on add_volume inside the loop.

https://bugzilla.gnome.org/show_bug.cgi?id=756589

gtkplacesview: remove dead code

This is checked on is_removable_volume

https://bugzilla.gnome.org/show_bug.cgi?id=756589

gtkplacessidebar: improve heuristics for external drives

Since the change to use GtkPlacesView we don't want to show
internal storage on the sidebar.

In our case we were checking for drive_can_eject and
drive_is_media_removable.

However for some external hard drives it's reported that they
are not ejectable nor the have removable media. So the only
attribute that they have different from internal drives is that
they can be stopped.
So check for if the drive can be stopped to decide if it is
external or internal.

On the way realized we don't need to check for the mounts associated
with the volume to know if the volume can be ejected or not. So remove
that code.

https://bugzilla.gnome.org/show_bug.cgi?id=756589

mir: add mircookie as a dependency

texthandle: Ensure handles are invalidated on mode changes

Otherwise the "cursor" handle stays with "cursor" appearance instead of
"selection end" when a text selection is started.

texthandle: small refactor

These long enums are used too often, shorten things a bit with temp vars.

texthandle: Fix Y positioning of text handles

It is assumed that border.top is the same than pointing_to.height (which
equals the strong cursor position), which is not since some time ago.

The border calculation has been move on top too, it is now used in the
Y position one, and doesn't depend on anything we calculate later.